Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce Resource Sharing Policy Management Component #416

Open
wants to merge 58 commits into
base: main
Choose a base branch
from

Conversation

BimsaraBodaragama
Copy link

@BimsaraBodaragama BimsaraBodaragama commented Nov 25, 2024

This PR adds a new component to manage resource-sharing policies between organizations in the WSO2 Identity Server. It enables defining, storing, and retrieving resource-sharing policies for users, applications, and identity providers.

Purpose

This feature addresses the need for a structured and scalable solution to manage resource-sharing policies across organizational hierarchies. It resolves the following issues:

Goals

  • Provide a unified interface for managing resource-sharing policies.
  • Enable consistent policy definitions across resource types.
  • Support dynamic and hierarchical sharing options, such as sharing with all current and future organizations or selected organizational hierarchies.

Approach

  • Created a new component for resource-sharing policy management.
  • Developed a core service with DAO and utility classes for CRUD operations.
  • Defined constants and enums for resource types and sharing policies.
  • Implemented robust exception handling mechanisms.

Release note

Introduced a centralized resource-sharing policy management service to unify and streamline sharing policies across all resource types.

Documentation

N/A (Documentation updates will be tracked separately.)

Test environment

  • JDK 11
  • macOS 15.1.1 (24B91)
  • H2 Database

Related Issue

Introduce a Centralized Resource Sharing Policy Management Service #21815

…licy record and shared resource attributes by resource sharing policy id
… attributes rather than deleting it from record id since that way of deletion is not currently needed.
… attributes rather than deleting it from record id since that way of deletion is not currently needed - Part 2.
@BimsaraBodaragama BimsaraBodaragama self-assigned this Nov 25, 2024
Copy link

codecov bot commented Nov 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.33%. Comparing base (8048fae) to head (36841d7).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #416      +/-   ##
============================================
+ Coverage     47.14%   49.33%   +2.19%     
- Complexity     1074     1170      +96     
============================================
  Files           125      128       +3     
  Lines          7021     7270     +249     
  Branches        849      858       +9     
============================================
+ Hits           3310     3587     +277     
+ Misses         3403     3378      -25     
+ Partials        308      305       -3     
Flag Coverage Δ
unit 35.59% <100.00%> (+4.55%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants